Fix avoiding a rebuild when moving around a workspace
authorAlex Crichton <alex@alexcrichton.com>
Fri, 29 Jun 2018 19:17:41 +0000 (12:17 -0700)
committerMark Rousskov <mark.simulacrum@gmail.com>
Fri, 29 Jun 2018 21:13:31 +0000 (15:13 -0600)
commit68572ee39488731df741693af67abaf3560b510f
treeb9dd77aae5ddcf7f1e3197cfa3fd02268bb062ae
parente2348c2db296ce33428933c3ab8786d5f3c54a2e
Fix avoiding a rebuild when moving around a workspace

There's a case where Cargo will recompile a project even if the fingerprint
looks like it's fresh, when some output files are missing. This was intended to
cover the case where an output file was deleted manually or otherwise messed
with. The check here was a bit too eager, however. It checked not only the
actual output destination of the compiler but *also* the location that we hard
link the output file up to.

Due to recent changes in #5460 we don't always create the hard links for path
dependencies in the top-level dir, and this meant that if the library were
compiled and then tested later on the test may recompile the original library by
accident.

The fix in this commit is to cease looking for the hardlink if it exists or not.
This way we only check for the presence of the output file itself and only
recompile if that file is missing. The reason for this is that we
unconditionally relink files into place whether it's fresh or not, so we'll
always recreate the hard link anyway if it's missing.

cc rust-lang/rust#51717
src/cargo/core/compiler/fingerprint.rs
tests/testsuite/freshness.rs